Skip to content

EIP-2935: Serve historical block hashes from state #2749

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 13 commits into from
May 8, 2025
Merged

EIP-2935: Serve historical block hashes from state #2749

merged 13 commits into from
May 8, 2025

Conversation

anvacaru
Copy link
Contributor

@anvacaru anvacaru commented Apr 24, 2025 •
edited
Loading

Implementation of EIP-2935.

blocked on:
#2747

Changes:

  • Added new EVM command #executeBlockHashHistory for EIP-2935 support.
  • Modified #startBlock rule to execute #executeBlockHashHistory after #executeBeaconRoots.
  • Introduced schedule flag Ghashistory to control EIP-2935 activation.
  • Updated schedule definitions and constants to include Ghashistory under Prague fork.
  • Implemented storage update logic for HISTORY_STORAGE_ADDRESS when Ghashistory flag is active.

@anvacaru anvacaru marked this pull request as ready for review May 8, 2025 06:09
@anvacaru anvacaru requested a review from palinatolmach May 8, 2025 06:09
@anvacaru anvacaru mentioned this pull request May 8, 2025
Open
palinatolmach
palinatolmach approved these changes May 8, 2025
View reviewed changes
Copy link
Contributor

@palinatolmach palinatolmach left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Left one minor comment.

@automergerpr-permission-manager automergerpr-permission-manager bot merged commit a11f28d into master May 8, 2025
12 checks passed
@automergerpr-permission-manager automergerpr-permission-manager bot deleted the eip-2935 branch May 8, 2025 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@palinatolmach palinatolmach palinatolmach approved these changes

Assignees
No one assigned
Labels
automerge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@anvacaru @palinatolmach